Skip to content

Change Bazel to use Clang for layering checks (backport #1706) - #1709

Merged
luca-della-vedova merged 1 commit into
sdf15from
mergify/bp/sdf15/pr-1706
Sep 10, 2026
Merged

Change Bazel to use Clang for layering checks (backport #1706)#1709
luca-della-vedova merged 1 commit into
sdf15from
mergify/bp/sdf15/pr-1706

Conversation

@mergify

@mergify mergify Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

🦟 Bug fix

Tracking issue: gazebo-tooling/gz-bazel-dev#5

Summary

Change bazel build to use clang

Backport Policy

These versions have a MODULE.bazel file

  • This is safe to backport to the following versions:
    • Jetty (sdf16)
    • Ionic (sdf15)
    • Harmonic
    • Fortress
  • This should not be backported
  • I am not sure
  • Other (fill in yourself)

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the fix (as needed)
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Assisted-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.


This is an automatic backport of pull request #1706 done by [Mergify](https://mergify.com).

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.55%. Comparing base (87b8625) to head (cfc1f6c).

Additional details and impacted files
@@           Coverage Diff           @@
##            sdf15    #1709   +/-   ##
=======================================
  Coverage   92.55%   92.55%           
=======================================
  Files         139      139           
  Lines       18651    18651           
=======================================
  Hits        17263    17263           
  Misses       1388     1388           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Cherie Hu <cheriehu@intrinsic.ai>
(cherry picked from commit 9cb95ac)
@luca-della-vedova
luca-della-vedova enabled auto-merge (squash) September 10, 2026 11:19
@github-project-automation github-project-automation Bot moved this from Inbox to In review in Core development Sep 10, 2026
@luca-della-vedova
luca-della-vedova merged commit edf2e09 into sdf15 Sep 10, 2026
16 checks passed
@luca-della-vedova
luca-della-vedova deleted the mergify/bp/sdf15/pr-1706 branch September 10, 2026 11:53
@github-project-automation github-project-automation Bot moved this from In review to Done in Core development Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏛️ ionic Gazebo Ionic

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants